Skip to content

chore: only run autofix on PRs#129

Merged
KevinVandy merged 1 commit into
mainfrom
autofix-config
May 21, 2026
Merged

chore: only run autofix on PRs#129
KevinVandy merged 1 commit into
mainfrom
autofix-config

Conversation

@KevinVandy
Copy link
Copy Markdown
Member

@KevinVandy KevinVandy commented May 21, 2026 •
edited by coderabbitai Bot
Loading

🎯 Changes

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

Summary by CodeRabbit

  • Chores
    • Updated the automated code fix workflow to trigger exclusively on pull requests rather than direct commits. This streamlines the continuous integration process by ensuring code improvements are reviewed during the pull request stage.

Review Change Stack

@KevinVandy KevinVandy requested a review from a team as a code owner May 21, 2026 21:35
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a685f044-0a54-4261-8a56-84d99f84593a

📥 Commits

Reviewing files that changed from the base of the PR and between 404666b and 219b753.

📒 Files selected for processing (1)
  • .github/workflows/autofix.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/autofix.yml
📝 Walkthrough

Walkthrough

The autofix workflow trigger configuration is simplified to remove the push event listener. The workflow now exclusively runs on pull_request events instead of automatically running on pushed commits to tracked branches.

Changes

Autofix Workflow Trigger

Layer / File(s) Summary
Pull request-only trigger configuration
.github/workflows/autofix.yml		 The workflow on trigger removes the push event (previously configured for main, alpha, beta, rc branches) and retains only the pull_request trigger.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The rabbit hops with glee, so spry,
As workflows now skip push—goodbye!
Pull requests alone shall trigger the dance,
A simpler path, a cleaner chance,
The autofix spirits now align,
In PR-only bliss, they surely shine!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing the push event trigger from the autofix workflow to only run on pull requests.
Description check ✅ Passed The PR description follows the required template structure with all sections present, but the 'Changes' section lacks detail explaining the motivation and rationale for the workflow modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch autofix-config

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

🚀 Changeset Version Preview

No changeset entries found. Merging this PR will not cause a version bump for any packages.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 21, 2026

Open in StackBlitz

@tanstack/angular-hotkeys

npm i https://pkg.pr.new/@tanstack/angular-hotkeys@129

@tanstack/hotkeys

npm i https://pkg.pr.new/@tanstack/hotkeys@129

@tanstack/hotkeys-devtools

npm i https://pkg.pr.new/@tanstack/hotkeys-devtools@129

@tanstack/lit-hotkeys

npm i https://pkg.pr.new/@tanstack/lit-hotkeys@129

@tanstack/preact-hotkeys

npm i https://pkg.pr.new/@tanstack/preact-hotkeys@129

@tanstack/preact-hotkeys-devtools

npm i https://pkg.pr.new/@tanstack/preact-hotkeys-devtools@129

@tanstack/react-hotkeys

npm i https://pkg.pr.new/@tanstack/react-hotkeys@129

@tanstack/react-hotkeys-devtools

npm i https://pkg.pr.new/@tanstack/react-hotkeys-devtools@129

@tanstack/solid-hotkeys

npm i https://pkg.pr.new/@tanstack/solid-hotkeys@129

@tanstack/solid-hotkeys-devtools

npm i https://pkg.pr.new/@tanstack/solid-hotkeys-devtools@129

@tanstack/svelte-hotkeys

npm i https://pkg.pr.new/@tanstack/svelte-hotkeys@129

@tanstack/vue-hotkeys

npm i https://pkg.pr.new/@tanstack/vue-hotkeys@129

@tanstack/vue-hotkeys-devtools

npm i https://pkg.pr.new/@tanstack/vue-hotkeys-devtools@129

commit: 219b753

@KevinVandy KevinVandy merged commit 6adcc3a into main May 21, 2026
9 checks passed
@KevinVandy KevinVandy deleted the autofix-config branch May 21, 2026 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schiller-manuel schiller-manuel schiller-manuel approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KevinVandy @schiller-manuel